Skip to content

Proget parameterizing#395

Merged
vipbhardwaj merged 7 commits into
mainfrom
proget-parameterizing
Apr 24, 2026
Merged

Proget parameterizing#395
vipbhardwaj merged 7 commits into
mainfrom
proget-parameterizing

Conversation

@vipbhardwaj
Copy link
Copy Markdown
Contributor

@vipbhardwaj vipbhardwaj commented Apr 23, 2026

CBG-5315

Using proget to parameterize the CBL version picking since its been a pain point for SGW jobs as they are run on a scheduled basis.

  • This picks the CBL version via proget always
  • Only SGW version via proget for sgw-functional pipeline.
  • For SGW upgrade pipelines SGW version upgrade paths have to be predefined hence are not picked via proget.

vipbhardwaj and others added 2 commits April 23, 2026 09:57
- SyncGatewayVersion.parse() and EdgeServerVersion.parse(): prevent
  empty version string (now returns 'unknown'), catch ValueError on
  non-numeric build numbers, add warning logs for unexpected formats
- SyncGateway.get_version() and EdgeServer.get_version(): replace
  assert with graceful fallback when version string lacks '/' separator,
  use rsplit to handle multiple slashes
- GreenboardUploader.upload(): accept CouchbaseVersion object instead
  of pre-formatted string to eliminate fragile serialize-then-reparse
  pattern; use .version/.build_number directly for SGW jobs
- greenboard_fixture.py: pass CouchbaseVersion object directly instead
  of constructing intermediate string
- Add unit tests for version parsing edge cases (14 tests)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Prelease versions are picked for CBL in SGW job
- For upgrade jobs CBL version only is picked since SGW has a series of defined versions and cannot resort to just picking prerelease
@vipbhardwaj vipbhardwaj requested a review from borrrden April 23, 2026 05:55
@vipbhardwaj vipbhardwaj merged commit 2af0db6 into main Apr 24, 2026
5 checks passed
@vipbhardwaj vipbhardwaj deleted the proget-parameterizing branch April 24, 2026 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants